Skip to content

test: add library doc comments#2586

Merged
kevmoo merged 2 commits intomasterfrom
test_basic
Feb 4, 2026
Merged

test: add library doc comments#2586
kevmoo merged 2 commits intomasterfrom
test_basic

Conversation

@kevmoo
Copy link
Copy Markdown
Member

@kevmoo kevmoo commented Jan 20, 2026

No description provided.

@kevmoo kevmoo requested a review from a team as a code owner January 20, 2026 06:24
@github-actions
Copy link
Copy Markdown

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

Comment on lines +11 to +12
/// This is a re-export of `package:test_api/fake.dart`. It is kept separate
/// to avoid a breaking change for users who import `package:mockito` directly.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is worth documenting. The test_api library is not user facing so it's not worth mentioning in user facing docs. The history of it's landing in the test package isn't really interesting for users either. I'm not convinced we'd want to necessarily export Fake from test/teat outside of Mockito considerations, but I agree it's the case that would fit in more with the existing library design.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's in the publicly visible, generated API docs. It's better than having folks guess!

@kevmoo kevmoo merged commit 5a2b40d into master Feb 4, 2026
60 checks passed
@kevmoo kevmoo deleted the test_basic branch February 4, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants